Skip to content

Conversation

@Kamalpannu
Copy link

Checklist
(Check off all the items before submitting)

  • Build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Purpose
Fix the issue with missing scrollbarStyles export so it can be imported correctly in index.ts.

Description

  • Added scrollbarStyles export in src/index.ts.
  • Verified TS build passes and website builds correctly.
  • Ensured all tests in /lib pass successfully.

Screenshots
Not applicable.

Additional context
The import ./styles/scrollbarStyles was missing, causing TS build failures in the library.

**Closes #2190 **

@Kamalpannu
Copy link
Author

Hi @GomezIvann, just checking in to see if you had a chance to review this PR.
It exports the scrollbar styles as suggested in #2190, following the design system conventions.
Happy to make adjustments or add documentation updates if needed!

@Jialecl
Copy link
Collaborator

Jialecl commented Nov 18, 2025

Hello @Kamalpannu,
Thank you for opening this PR. We appreciate it and will review and discuss it as soon as we have some time.

Comment on lines +37 to +39
"dependencies": {
"@dxc-technology/halstack-react": "^0.0.0"
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be added.

export { default as DxcWizard } from "./wizard/Wizard";
export { HalstackProvider, HalstackLanguageContext } from "./HalstackContext";
export { default as useToast } from "./toast/useToast";
export { default as scrollbarStyles } from "./styles/scroll";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a valid idea, however we should update the documentation to let the user know that he can import this mixin for his custom components. I will check where we can insert this information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow users to use halstack scroll styles

3 participants